feat: (poc) Unified AI Search#40890
Conversation
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
🦋 Changeset detectedLatest commit: 608bc6d The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
# Conflicts: # apps/meteor/client/navbar/NavBarSearch/NavBarSearch.tsx # apps/meteor/client/navbar/NavBarSearch/NavBarSearchItem.tsx # apps/meteor/client/navbar/NavBarSearch/NavBarSearchListbox.tsx # apps/meteor/server/methods/messageSearch.ts # apps/meteor/tests/end-to-end/api/miscellaneous.ts # packages/core-services/src/index.ts
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #40890 +/- ##
===========================================
+ Coverage 69.96% 69.98% +0.01%
===========================================
Files 3353 3364 +11
Lines 128820 130345 +1525
Branches 22294 22655 +361
===========================================
+ Hits 90132 91225 +1093
- Misses 35397 35790 +393
- Partials 3291 3330 +39
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
✅ Layne — scan passed No security issues found on latest push. |
| const response = await fetch(`${trimTrailingSlashes(provider.baseUrl)}/chat/completions`, { | ||
| method: 'POST', | ||
| timeout: 20000, | ||
| ignoreSsrfValidation: true, |
There was a problem hiding this comment.
Could you add a ´// SECURITY: here and in otherignoreSsrfValidation: true` just so we can have it registered the reason why it's being disabled? Then I can approve it as an exception on Layne. Thank you!
There was a problem hiding this comment.
Sure, thank you for the review.
There was a problem hiding this comment.
removed that, it was no longer needed
3 participants
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments
AI-53